Reapply HDF reader fillvalue#460
Conversation
Use context manager in tests to ensure resources are released.
|
@TomNicholas, @maxrjones, @sharkinsspatial, I believe this fixes the issue with the hanging unit tests that started when #420 was merged. I have reverted Tom's revert in #457 (thus restoring the origin changes from #420) and have updated related unit tests to use context managers so that file resources are cleaned up, and this appears to have fixed the problem. Using the origin merge of #420, I was able to periodically (not consistently; perhaps every 3rd attempt) reproduce the hanging test issue locally via |
|
Amazing, thank you @chuckwondo ! |
|
Thanks @chuckwondo. 🙇 |
Reapply reverted changes from #420, but with tests modified to use context managers to cleanup resources in an attempt to prevent the build from hanging during unit tests.